Skip to content

Conversation

@WeylonSantana
Copy link
Contributor

@WeylonSantana WeylonSantana commented Mar 12, 2025

assets - AscensionGameDev/Intersect-Assets#68

1741809564_Intersect_Client.mp4

Copy link
Member

@pandinocoder pandinocoder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if there are other instances of the ! thing, but if there were any of those in this PR that I missed they should be changed too

LoadJsonUi(GameContentManager.UI.InGame, Graphics.Renderer.GetResolutionString());

_contextMenu.ClearChildren();
_contextMenu!.ClearChildren();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ? not !

{
// Clear out the old options.
_contextMenu.ClearChildren();
_contextMenu!.ClearChildren();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ? not !


//TODO: Is this a memory leak?
_contextMenu.ClearChildren();
_contextMenu!.ClearChildren();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ? not !

{
// Clear out the old options since we might not show all of them
_contextMenu.ClearChildren();
_contextMenu!.ClearChildren();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ? not !

LoadJsonUi(GameContentManager.UI.InGame, Graphics.Renderer.GetResolutionString());

_contextMenu.ClearChildren();
_contextMenu!.ClearChildren();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be ? not !

@pandinocoder pandinocoder merged commit c44b05b into AscensionGameDev:main Mar 12, 2025
1 check passed
@pandinocoder pandinocoder deleted the hotbar branch March 12, 2025 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants